Skip to content

Remove use of AWS stage server#4652

Merged
julienw merged 2 commits into
firefox-devtools:mainfrom
willkg:patch-2
Jun 6, 2023
Merged

Remove use of AWS stage server#4652
julienw merged 2 commits into
firefox-devtools:mainfrom
willkg:patch-2

Conversation

@willkg

@willkg willkg commented May 31, 2023

Copy link
Copy Markdown
Contributor

This removes the host that pointed to the stage environment in AWS for the Mozilla Symbolication Service. That host no longer works.

See https://bugzilla.mozilla.org/show_bug.cgi?id=1836055

This removes the host that pointed to the stage environment in AWS for the Mozilla Symbolication Service. That host no longer works.

See https://bugzilla.mozilla.org/show_bug.cgi?id=1836055
// -----------------
// Can be overridden with the URL parameter `symbolServer=SERVERURL`.
// You can change this to the staging server `https://symbolication.stage.mozaws.net`,
// or run a local symbol server (for example using profiler-symbol-server [1])

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Further, the SYMBOL_SERVER_URL constant below should be renamed to SYMBOLICATION_SERVICE_URL since the symbol server is a different thing.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, looks like we got used to misname this :-)

@codecov

codecov Bot commented May 31, 2023

Copy link
Copy Markdown

Codecov Report

Patch and project coverage have no change.

Comparison is base (acb9e66) 88.47% compared to head (f5d43f0) 88.47%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4652   +/-   ##
=======================================
  Coverage   88.47%   88.47%           
=======================================
  Files         295      295           
  Lines       26213    26213           
  Branches     7068     7068           
=======================================
  Hits        23193    23193           
  Misses       2809     2809           
  Partials      211      211           
Impacted Files Coverage Δ
src/app-logic/constants.js 100.00% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@julienw julienw enabled auto-merge (squash) June 6, 2023 16:30
@julienw julienw disabled auto-merge June 6, 2023 16:30
@julienw julienw enabled auto-merge (squash) June 6, 2023 16:31
@julienw julienw merged commit 0410831 into firefox-devtools:main Jun 6, 2023
@willkg

willkg commented Jun 12, 2023

Copy link
Copy Markdown
Contributor Author

Thank you!

@canova canova mentioned this pull request Jun 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants